Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Failover verification #30

Merged
merged 3 commits into from
Jan 4, 2023
Merged

Failover verification #30

merged 3 commits into from
Jan 4, 2023

Conversation

davissp14
Copy link
Contributor

@davissp14 davissp14 commented Jan 4, 2023 •
edited
Loading

Custom failover validation process that will prevent failovers when quorum can't be met.

Notable impact:

  • This will help mitigate certain split-brain scenarios that would occur during a network split.
  • This will remove HA from a 2-node setup until we can properly differentiate a down node from a network split.
lax [info]repmgrd         | [2023-01-04 21:19:20] [DETAIL] /usr/local/bin/failover_validation -visible-nodes 1 -total-nodes 2
lax [info]repmgrd         | [2023-01-04 21:19:20] [INFO] output returned by failover validation command:
lax [info]repmgrd         | Unable to perform failover as quorum can not be met. Total nodes: 2, Visible nodes: 1

@davissp14 davissp14 merged commit 4fd213a into master Jan 4, 2023
@davissp14 davissp14 deleted the failover-verification branch February 25, 2023 01:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DAlperin DAlperin Awaiting requested review from DAlperin

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@davissp14